@W-19978113 Build Core Runtime Error Capture and Diagnostic Payload / Implement Error Handling System#543
Open
deepu-mungamuri94 wants to merge 2 commits intosalesforcecli:mainfrom
Conversation
|
Thanks for the contribution! Unfortunately we can't verify the commit author(s): dmungamuri <d***@s***.com>. One possible solution is to add that email to your GitHub account. Alternatively you can change your commits to another email and force push the change. After getting your commits associated with your GitHub account, refresh the status of this Pull Request. |
1effd82 to
8365c38
Compare
@W-19978113@ This adds a robust error capture and reporting system for LWC local development that captures runtime errors from the browser and displays them formatted in the terminal. Features: - ErrorStore: In-memory store for runtime errors with deduplication - Error Capture HTTP Server: Standalone server on LWC port + 1 - Error Middleware: Express middleware for capturing/querying errors - Error Formatter: CLI-friendly formatted error output with colors - Stack Trace Utils: Parse and sanitize stack traces Components: - src/lwc-dev-server/errorStore.ts: Error storage with statistics - src/lwc-dev-server/errorHttpServer.ts: HTTP server for error endpoints - src/lwc-dev-server/errorMiddleware.ts: Express middleware - src/shared/errorFormatter.ts: Format errors for terminal display - src/shared/stackTraceUtils.ts: Stack trace parsing utilities - src/types/errorPayload.ts: TypeScript types for error diagnostics Testing: - Comprehensive unit tests for all error capture components - E2E tests for error capture workflow - Test fixtures for validation Updated: - package.json: Added express dependency - src/lwc-dev-server/index.ts: Integrated error capture server - yarn.lock: Updated dependencies Error Capture Endpoints: - POST /_dev/errors - Capture error reports - GET /_dev/errors - Query errors with filters - DELETE /_dev/errors - Clear all errors - GET /_dev/errors/stats - Get error statistics - GET /_dev/health - Health check Auto-clears errors on server restart. Can also manually clear via DELETE endpoint.
8365c38 to
d95fc3b
Compare
… logs - Remove verbose console logs on error capture server startup - Add server-side cleanup for _clientParsedStack field (client-only data) - Keep filtered stack traces (show local source only, hide framework code) - Improve code documentation and comments
a31362f to
5d0975f
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
@W-19978113@
What does this PR do?
This adds a robust error capture and reporting system for LWC local development that captures runtime errors from the browser and displays them formatted in the terminal.
Features:
Components:
Testing:
Updated:
Error Capture Endpoints:
Auto-clears errors on server restart. Can also manually clear via DELETE endpoint.
What issues does this PR fix or reference?